-
Notifications
You must be signed in to change notification settings - Fork 2.8k
[ZEPPELIN-1323] Add contribution guide for Zeppelin documentation #1326
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
CI failure looks unrelated to this change. Looks great to me! |
docs/CONTRIBUTING.md
Outdated
| If you're going to create new pages, there are some spots you need to add the location of the page. | ||
|
|
||
| - **Dropdown menu in navbar**: add your docs location to [_navigation.html](https://github.com/apache/zeppelin/blob/master/docs/_includes/themes/zeppelin/_navigation.html) | ||
| - **Main index**: add your docs below [What is the next?](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/#what-is-the-next) section in [index.md](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/#what-is-the-next) with short description. No need to do this if the page is for **Interpreters**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this point to http://zeppelin.apache.org/docs/latest?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felixcheung Yeah right. Thx for pointing it out!
281e94e to
2879ada
Compare
|
LGTM |
|
Is this good to go now? :) |
|
Merging if there is no more discussion.
|
What is this PR for?
Recently I added auto generated Table of Contents by #1031 and searching feature by #1266. So I wrote some guide to
docs/CONTRIBUTING.mdabout how to use and how they can be applied to docs page. Hope this helps to new Zeppelin (docs) contributors :)What type of PR is it?
Documentation
Todos
docs/README.mdWhat is the Jira issue?
ZEPPELIN-1323
How should this be tested?
Just click
Viewbutton on the top of the file diff :)Screenshots (if appropriate)
Questions: